Fixes for MPI-ESM1-2-HR and MPI-ESM1-2-XR#356
Conversation
|
thanks a bunch @sloosvel 🍺 Note that |
|
I added the institute and activity manually in the recipes where I use the model, but sure I can add it to the config-developer if you prefer. |
|
well we need something that's foolproof, ie not something the user would add or forget to add to their recipes, ideally the CV should be updated centrally, but in the absence of that config-developer would do 🍺 |
|
ooh can you also add |
|
Sure, even though in #107 (comment) @jvegasbsc was against doing that. But whatever you prefer. |
| input_file: '{short_name}_{mip}_{dataset}_{exp}_{ensemble}_{grid}*.nc' | ||
| output_file: '{project}_{dataset}_{mip}_{exp}_{ensemble}_{short_name}_{start_year}-{end_year}' | ||
| institutes: | ||
| 'CESM2-WACCM': ['NCAR'] |
There was a problem hiding this comment.
This should not be needed if we just update the cmor tables to the latest version. I made a pull request here: #363
There was a problem hiding this comment.
So this list will have to be updated every time the tables get updated?
There was a problem hiding this comment.
I think the other institute can also be removed.
There was a problem hiding this comment.
careful what you merge first - if you merge this PR before #363 then it won't work proper 🍺
6 participants
Before you start, please read CONTRIBUTING.md.
Tasks
yamllintto check that your YAML files do not contain mistakesIf you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.
Closes {#355}